fix(stories): use Space for vertical stacks of inline-level items#1168
Conversation
Replace Flow with Space flow="column" in stories where children are inline-level (ItemButton/Item/Button/Text), so vertical stacking and gap actually apply. Flow stays block-flow as designed. Co-authored-by: Cursor <cursoragent@cursor.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
📦 NPM canary releaseDeployed canary version 0.0.0-canary-efb338d. |
🏋️ Size limit report
Click here if you want to find out what is changed in this build |
🧪 Storybook is successfully deployed!
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 653dc73. Configure here.
1 participant
Replace Flow with Space flow="column" in stories where children are inline-level (ItemButton/Item/Button/Text), so vertical stacking and gap actually apply. Flow stays block-flow as designed.
Describe changes
Checklist
Before taking this PR from the draft, please, make sure you have done the following:
Closes: N/A
Other information
Note
Low Risk
Low risk because changes are limited to Storybook layout markup and imports, with no production component logic modified.
Overview
Storybook examples for
ItemAction,ItemButton,Menu,Item,ComboBox,FilterPicker,ListBox,TextArea, andTabsreplaceFlowwrappers withSpace flow="column"(orSpace flow="row"/"row wrap") where children are inline-level, ensuring vertical stacking andgapspacing apply as intended.Also adjusts related story layout details (imports, a few wrapper props, and removing manual
marginspacing inTextAreastories in favor ofSpacegaps).Reviewed by Cursor Bugbot for commit 653dc73. Bugbot is set up for automated code reviews on this repo. Configure here.